Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Docker-based tests for proxy connection mode #51658

Merged
merged 2 commits into from
Feb 3, 2020

Conversation

ywelsch
Copy link
Contributor

@ywelsch ywelsch commented Jan 30, 2020

Adds more tests for the new "proxy" remote cluster connection mode, using a Docker-based setup, as well as testing SNI-based routing using HAProxy.

  • Checks that the new proxy mode can work in situations where the publish host of the nodes in the remote cluster are not routable.
  • Checks that the new proxy mode can work with clusters where nodes are hidden behind HAProxy.
  • Checks that the new proxy mode can work with clusters where nodes are hidden behind HAProxy, using SNI to identify the nodes/cluster behind HAProxy.

Relates #49067

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Network)

qa/remote-clusters/build.gradle Show resolved Hide resolved
qa/remote-clusters/build.gradle Show resolved Hide resolved
@ywelsch ywelsch requested a review from mark-vieira January 31, 2020 09:36
Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM 👍

Copy link
Contributor

@Tim-Brooks Tim-Brooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ywelsch ywelsch merged commit 10b7ffa into elastic:master Feb 3, 2020
ywelsch added a commit that referenced this pull request Feb 3, 2020
Adds more tests for the new "proxy" remote cluster connection mode, using a Docker-based
setup, as well as testing SNI-based routing using HAProxy.

- Checks that the new proxy mode can work in situations where the publish host of the nodes in
the remote cluster are not routable.
- Checks that the new proxy mode can work with clusters where nodes are hidden behind
HAProxy.
- Checks that the new proxy mode can work with clusters where nodes are hidden behind
HAProxy, using SNI to identify the nodes/cluster behind HAProxy.

Relates #49067
ywelsch added a commit that referenced this pull request Feb 3, 2020
Adds more tests for the new "proxy" remote cluster connection mode, using a Docker-based
setup, as well as testing SNI-based routing using HAProxy.

- Checks that the new proxy mode can work in situations where the publish host of the nodes in
the remote cluster are not routable.
- Checks that the new proxy mode can work with clusters where nodes are hidden behind
HAProxy.
- Checks that the new proxy mode can work with clusters where nodes are hidden behind
HAProxy, using SNI to identify the nodes/cluster behind HAProxy.

Relates #49067
@polyfractal polyfractal added v7.6.0 and removed v7.6.1 labels Feb 7, 2020
@dnhatn
Copy link
Member

dnhatn commented Feb 12, 2020

@ywelsch The new task, which was introduced in this PR, recently failed at https://gradle-enterprise.elastic.co/s/wy2g72s4z4kzy. Can you take a look?

@mark-vieira
Copy link
Contributor

@dnhatn that is this issue: #48027

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants